-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TestFailure] Resolving exceptions thrown across multiple GraphBolt
tests.
#7852
Conversation
@chang-l : In addition to the ones I’ve already fixed and the two you mentioned, there are 14 more similar torch.load calls (refer to the attached file). Would you like me to address all of them in this PR, or would that be too much for a single submission? |
I think it is fine to address them all in this PR. Also, would you mind to refer this PyTorch PR: pytorch/pytorch#137602 in your PR description? |
fix the lint please |
please run the tests in your local manually. |
@Rhett-Ying This PR passed our CI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rhett-Ying Yes, all unit tests are passed.
Description
This PR resolves the following exceptions appearing in multiple
GraphBolt
tests after merging of Pytorch PR#137602 (Flip default on weights_only):Checklist
Please feel free to remove inapplicable items for your PR.
Changes